feat(esp32): throttle Wi-Fi TX during Matter BDX OTA (v1.1.5) + subscription-log ANSI fix - #84
Merged
Merged
Conversation
matter-server colourises its logs, so a `\x1b[0m` reset sits between '<Node:N>' and the message. The anchored '<Node:N> (Re-)?Subscription succeeded' pattern never matched, hard-failing check_subscription_log on otherwise-healthy flashes (node 35 v1.1.4: version confirmed, node available + subscribed, gate still errored). Strip ANSI first in both scripts; also bring esp32-release.sh up to the ota-release.sh behaviour (match Re-Subscription, tail -1 the newest, poll for the late resubscribe). Assisted-by: AI
The #12 brownout mitigation (drop TX power for the concurrent flash-write + Wi-Fi-TX window of an OTA) only covered the HTTP break-glass path; the PRIMARY Matter (BDX) OTA path ran at the full 20 dBm ceiling. Swap in a HisenseOTARequestorDriver that lowers TX on idle-exit (update starting) and restores it on idle-enter (back to idle on any path: done/abort/timeout), registered via esp_matter_ota_requestor_set_config right after esp_matter::start(). Capture-once + restore-on-every-idle-enter so the node never sticks at 10 dBm. Not a substitute for the bulk cap; it lowers brownout probability during the OTA window (node-35 phy_init brownout). PROJECT_VER 1.1.4 -> 1.1.5 (int 10105). Built + delta-OTA flashed to node 35, verified live: 1.1.5, subscribable, diagnostics reading (CompressorHz, Features1/Faults1 valid). Assisted-by: AI
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two reliability changes for the ESP32 esp-matter node, both prompted by the node-35
phy_initbrownout investigation (#83).1. Throttle Wi-Fi TX during a Matter (BDX) OTA (
feat, v1.1.5)The #12 brownout mitigation drops Wi-Fi TX power for the concurrent flash-write + TX window of an OTA, but it only covered the HTTP break-glass path (
https_ota_task). The primary Matter (BDX) OTA path ran the whole download+apply at the full 20 dBm ceiling.HisenseOTARequestorDriver : ExtendedOTARequestorDriverlowers TX onHandleIdleStateExit()(update starting) and restores onHandleIdleStateEnter()(back to idle on any path: done/abort/timeout).esp_matter_ota_requestor_set_config()right afteresp_matter::start()(before the asynckDnssdInitializedauto-start consumes the driver).HISENSE_OTA_TX_POWER_QDBM(10 dBm). Not a substitute for the bulk cap; it lowers brownout probability during the OTA window.PROJECT_VER1.1.4 -> 1.1.5 (int 10105).2. Strip ANSI before the subscription-log grep (
fix)matter-server colourises its logs, so a
\x1b[0mreset sits between<Node:N>and the message. The anchored<Node:N> (Re-)?Subscription succeededpattern never matched, hard-failingcheck_subscription_logon otherwise-healthy flashes (node 35 v1.1.4: version confirmed, node available + subscribed, gate still errored). Strip ANSI first in bothesp32-release.shandota-release.sh; also bring the esp32 script up to the ameba behaviour (matchRe-Subscription,tail -1the newest, poll for the late resubscribe).Validation
SoftwareVersionString=1.1.5(int 10105) sustained; re-interviewed; subscribable.check_subscription_lognow passes (matter-server log confirms: <Node:35> Subscription succeeded).Assisted-by: AI